Замена всех proc_holder на /datum/action#9069
Conversation
|
Кстати, для локализаторов которые это увидят. https://discord.com/channels/617003227182792704/1334498548036800552/1498000956829208817 |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
Аналитический отчёт от Большого Брата. Обработка запроса завершена. Приветствую! Я проанализировал предложенные Вами изменения. С технической точки зрения, работа выглядит требующей доработок. Пулл-реквест выполняет масштабный рефакторинг систем магии и заклинаний, перенося логику обета молчания на систему трейтов и упорядочивая сигналы. Однако в коде обнаружен ряд критических ошибок: логика проверок для мимов во многих местах инвертирована, что блокирует корректную работу способностей. Также выявлены регрессии в логировании заклинаний из-за удаления аргумента forced, несоответствие в именовании трейта TRAIT_EMOTE_MUTE стандартам Paradise и изменение сигнатуры метода try_invoke. Необходимо исправить логические условия и восстановить параметры вызовов для обеспечения стабильности систем станции.
| #define COMSIG_WHEELCHAIR_BELL_RANG "wheelchair_bell_rang" | ||
|
|
||
| /// From /datum/action/cooldown/spell/touch/lay_on_hands/proc/determine_if_this_hurts_instead(), sent to the /mob/living/carbon/hurtguy: (/mob/living/carbon/mendicant) | ||
| #define COMSIG_ON_LAY_ON_HANDS "mob_ability_lay_on_hands" |
There was a problem hiding this comment.
Касаемо всех этих сигналов. Если ты добавляешь сигнал, он должен где-то отправляться. Мертвые сигналы - выстрел в хуй.
There was a problem hiding this comment.
Я пока добавлял весь файл с сигналом магии, если после переноса не нужны будут удалю
github-actions
Bot
added
the
Merge Conflict
добавил папку spell_types и все базовые типы спеллов оттуда
github-actions
Bot
added
Merge Conflict
Куча фиксов линтера + поменял /obj/item/melee/touch_attack на /obj/item/melee/magic_hand чтобы не было конфликта с god_hand.dm, потом верну назад
github-actions
Bot
added
Merge Conflict
заменил все заклинания с контрактов дьявола
перенес ТГ вариант чтобы не заебываться
пачка touch спеллов, небольшие исправления а также род форм
github-actions
Bot
added
Изменение карты
github-actions
Bot
removed
the
Изменение карты
добавил заклинания с лаваленда(кроме лечащего касания)
форма животного, целебное касание, майндсвап, аппендиксы, искажение пространства, грейт кнок
github-actions
Bot
removed
the
Изменение карты
github-actions
Bot
added
the
Изменение карты
github-actions
Bot
removed
the
Изменение карты
github-actions
Bot
added
the
Merge Conflict
2 participants
Что этот ПР делает
Заменяет все прок холдеры на нормальные экшны. Пока что ПР в драфте, в первом коммите добавлен только action/cooldown/spell, перенос будет в последующих коммитах
Почему это хорошо для игры
Очистка от старого говна.
Список изменений
🆑
refactor: прок холдеры заменены на экшны
refactor: маска мима теперь работает через трейт
refactor: заклинание для отмены требования одежды мага теперь активируется и выдает трейт вместо пассивной работы
refactor: заклинание мутации теперь работает через статус эффект
tweak: tesla blast с ТГ вместо молнии мага
tweak: watcher's look работает через ПКМ вместо альт клика
/:cl: